Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add JSON-schema export (KEP-35 part 2) #1727

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Conversation

ANeumann82
Copy link
Member

What this PR does / why we need it:
Add support for output types in kudo package list parameters:

kudo package list parameters --output yaml --> Outputs the package format of params.yaml
kudo package list parameters --output json-schema --> Outputs the parameters converted to JSON-Schema

Signed-off-by: Andreas Neumann <[email protected]>
Signed-off-by: Andreas Neumann <[email protected]>

# Conflicts:
#	pkg/kudoctl/cmd/package_list_params.go
Make sure topLevelGroups are required if they have required properties without defaults
Updated generated files

Signed-off-by: Andreas Neumann <[email protected]>
Signed-off-by: Andreas Neumann <[email protected]>
Signed-off-by: Andreas Neumann <[email protected]>
Adjusted json schema for schemas
Added tests

Signed-off-by: Andreas Neumann <[email protected]>
Signed-off-by: Andreas Neumann <[email protected]>
Signed-off-by: Andreas Neumann <[email protected]>
@ANeumann82 ANeumann82 marked this pull request as ready for review October 30, 2020 12:47
Signed-off-by: Andreas Neumann <[email protected]>

# Conflicts:
#	config/crds/kudo.dev_operatorversions.yaml
#	pkg/kudoctl/cmd/testdata/deploy-kudo-ns.yaml.golden
#	pkg/kudoctl/cmd/testdata/deploy-kudo-sa.yaml.golden
#	pkg/kudoctl/cmd/testdata/deploy-kudo.json.golden
#	pkg/kudoctl/cmd/testdata/deploy-kudo.yaml.golden
#	pkg/kudoctl/kudoinit/crd/bindata.go
Signed-off-by: Andreas Neumann <[email protected]>
@kensipe
Copy link
Member

kensipe commented Mar 12, 2021

@ANeumann82 is this ready for review?

@ANeumann82
Copy link
Member Author

@ANeumann82 is this ready for review?

It's been a while, but yeah, I think this was ready for review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants